Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix] stabilize the ODS-1864 test by scrolling button into view #1526

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

jstourac
Copy link
Member

@jstourac jstourac commented Jun 13, 2024

This stabilizes the Verify User Can Create A Workbench With Environment Variables test - it usually pass just fine locally, but fails on
Jenkins since the button isn't clickable as it's not in the view of the
browser.

This tries to move the button into the center of the view.

Also, unnecessary version check removed.

CI: /job/rhods/job/rhods-sanity/1682
image

@jstourac jstourac self-assigned this Jun 13, 2024
@jstourac jstourac added verified This PR has been tested with Jenkins enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) labels Jun 13, 2024
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
476 0 0 476 100

This stabilizes the `Verify User Can Create A Workbench With Environment
Variables` test - it usually pass just fine locally, but fails on
Jenkins since the button isn't clickable as it's not in the view of the
browser.

This tries to move the button into the center of the view.

Also, unnecessary version check removed.
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@jstourac jstourac marked this pull request as ready for review June 13, 2024 12:37
... //section[@aria-label="Environment variables"]/div[contains(@class, "-l-split")][${idx}]//button[@data-testid="another-key-value-pair-button"] # robocop: disable:line-too-long
# Assure that the button we want to click is in the view
Execute Javascript
... document.evaluate('${another_key_button_xpath}', document, null, XPathResult.FIRST_ORDERED_NODE_TYPE, null).singleNodeValue.scrollIntoView() # robocop: disable:line-too-long
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm, that is good, no? 🙂

@jstourac jstourac requested a review from jgarciao June 13, 2024 15:00
@jstourac jstourac merged commit cb466a2 into red-hat-data-services:master Jun 14, 2024
8 checks passed
@jstourac jstourac deleted the stabilizeOds1864 branch June 14, 2024 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants